test: add comprehensive SDK test coverage #398
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-Authored-By: Pedro Cunha <[email protected]>
Co-Authored-By: Pedro Cunha <[email protected]>
…irprotocol/relay-sdk into devin/1733868548-add-tvm-support
Co-Authored-By: Pedro Cunha <[email protected]>
- Add tests for cross-chain operations - Add tests for error recovery scenarios - Add tests for concurrent operations - Add tests for wallet implementations - Improve test coverage for critical flows Co-Authored-By: Pedro Cunha <[email protected]>
Contributor
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
🦋 Changeset detectedLatest commit: 9f3dc8e The changes in this PR will be included in the next version bump. This PR includes changesets to release 6 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Contributor
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
⚙️ Control Options:
Add "(aside)" to your comment to have me ignore it. |
Co-Authored-By: Pedro Cunha <[email protected]>
…rovements - Remove Gate.io wallet test pages and types - Implement proper chain switching in execute.ts - Fix gas error handling in transaction.ts - Clean up wallet adapter implementation - Update test suite with comprehensive coverage - Remove TVM-specific code Co-Authored-By: Pedro Cunha <[email protected]>
Co-Authored-By: Pedro Cunha <[email protected]>
Co-Authored-By: Pedro Cunha <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
SDK Test Coverage Improvements and Gate Wallet Removal
Changes
Test Coverage Improvements
Cross-chain operations testing
Error recovery scenarios
Concurrent operations
Wallet implementations
Testing Status
All tests passing (60 tests across 7 test files)
Linting checks passed
Build verification completed
Link to Devin run: https://app.devin.ai/sessions/fa3ded7a8cc14e099573e465f6c0e8b6